Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XWIKI-13967: Confusion between Supported Languages and Default Languages when multilingual is set to no #2770

Merged
merged 2 commits into from
Jan 18, 2024

Conversation

Sereza7
Copy link
Contributor

@Sereza7 Sereza7 commented Jan 4, 2024

Jira

https://jira.xwiki.org/browse/XWIKI-13967

PR Changes

  • Added a CSS ruleset to remove the supported languages field from the UI when it's not useful.

Note

This makes use of the new CSS feature :has, which is only available in the latest stable release of Firefox. For now this release is not used by all users but in less than 30 days we can count on 70% of users moving on this version. All of the other supported browser for XWiki have supported :has for a while. It's a correct change in regards to our browser support policy.
<EDIT> On older versions of Firefox, the rule is ignored and the supported language field is always shown. </EDIT>

View

Here is a demo of the changes brought in this PR. We can see that the support languages field disappears when the wiki is not set as multilingual.

Tests

Successfully passed mvn clean install -f xwiki-platform-core/xwiki-platform-administration/xwiki-platform-administration-test/xwiki-platform-administration-test-docker -Dxwiki.enforcer.skip=true -Dgradle.cache.local.enabled=false -Dgradle.cache.remote.enabled=false.

Sereza7 and others added 2 commits January 4, 2024 15:49
…ges when multilingual is set to no

* Added a CSS ruleset to remove the supported languages field from the UI when it's not useful.
@manuelleduc
Copy link
Contributor

Out of curiosity, why do you need those options in the maven command: -Dxwiki.enforcer.skip=true -Dgradle.cache.local.enabled=false -Dgradle.cache.remote.enabled=false ?

@Sereza7
Copy link
Contributor Author

Sereza7 commented Jan 18, 2024

@manuelleduc

  • -Dxwiki.enforcer.skip=true at this time, I didn't upgrade my maven yet, so the enforcer failed for any build and I disabled it
  • -Dgradle.cache.local.enabled=false -Dgradle.cache.remote.enabled=false to make sure I run everything and don't rely on the cache. I don't understand those options very well but IMO they just disable some build cache, which can not be a bad thing for tests. It's slower but at least I'm sure to test the actual changes brought in the PR.

@surli surli added the backport stable-15.10.x Used for automatic backport to 15.10.x branch. label Jan 18, 2024
@surli surli merged commit fcb4699 into xwiki:master Jan 18, 2024
1 check passed
github-actions bot pushed a commit that referenced this pull request Jan 18, 2024
…ges when multilingual is set to no (#2770)

* Added a CSS ruleset to remove the supported languages field from the UI when it's not useful.

(cherry picked from commit fcb4699)
surli pushed a commit that referenced this pull request Jan 18, 2024
…ges when multilingual is set to no (#2770) (#2805)

* Added a CSS ruleset to remove the supported languages field from the UI when it's not useful.

(cherry picked from commit fcb4699)

Co-authored-by: C. Lucas <[email protected]>
Sereza7 added a commit to Sereza7/xwiki-platform that referenced this pull request Jan 18, 2024
…ges when multilingual is set to no (xwiki#2770)

* Added a CSS ruleset to remove the supported languages field from the UI when it's not useful.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport stable-15.10.x Used for automatic backport to 15.10.x branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants